Description

Move chi.RegisterMethod("REPORT") from webdav.NewService into a package init() so it runs before any service goroutine is started by the suture supervisor.

Related Issue

• Fixes Settings Service crashes during HTTP server initialization with a Go runtime panic in chi route registration #2686

Motivation and Context

chi.RegisterMethod mutates the package-global methodMap in github.com/go-chi/chi/v5, which (*node).setEndpoint iterates during route insertion. opencloud starts many services concurrently under suture, so calling RegisterMethod inside NewService could race with another service's mux.Route(...) and trigger a "concurrent map iteration and map write" fatal (the stack in #2686). Doing it in init() happens during package import, before any supervised goroutine starts. chi.RegisterMethod is idempotent so it stays safe.

How Has This Been Tested?

• go build ./... and gofmt on the affected package pass.
• go test -race ./services/webdav/pkg/service/v0/... builds cleanly under the race detector.

I do not have a deterministic local reproducer for the intermittent panic, but the race is visible in the chi source (a package-global map written by RegisterMethod and iterated by setEndpoint).

Screenshots (if appropriate):

n/a

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests only (no source changes)

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation added (changelog entry)